Квоты для профессий 2.0 #13632
Квоты для профессий 2.0 #13632
Conversation
Changelog status: ✔️ |
|
Изначально я хотел перевести консоль на тгуи, но он у меня до конца не починился, потому, пришлось изучать и копаться в nanoui. Зато было познавательно, узнал что такое dot.js, покопался в файлах и пр. ПР сделан по результатам голосования на форуме. |
|
Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности: https://forum.taucetistation.org/t/loupop-i-reshenie-problemy-resov/44068/105 |
|
Эм, ограничение сделай на количество. 1 профа из отдела или всего 3 профы глобально. И минусовать профы нет смысла, всё равно тот кто хочет зайти за библиотекаря ненужного и так зайдёт, за офицера ненужного также, но про игроков на ГП думать будет плохо в дальнейшем |
ресы есть, шахтёр не нужен Чтобы не было такого что ты заходишь на профу, а твоё место работы уже заняли и ты - повторка. |
|
Чтобы уменьшить мисскомуникейшен, можно на каждый лайк и дизлайк требовать от ГП ввести короткий текст, который отобразится в меню лейтджоина рядом с профой. Что я имею ввиду: |
это уже иквок передача текста в лобби... Уже вижу ГП что пишет про нюку и лайкает сбух А сейчас, увидел лайк на сбухе и думаешь, а почему гп его лайкнул... они нужны? А почему? Надо зайти посмотреть, возьму сбуху. |
|
ещё можно что если зашел за нежелаемую профу то минусы к зарплате а если за желаемую то плюсы |
ну хз, потом можно будет. Сейчас это - просто механ. а там посмотреть надо как будет работать на людях. |
|
Какая-то страшная кнопка: |
|
Цветовая индикация создаёт ощущение, что вход за эти профы заблокирован (что в целом было бы прикольно, но негодяи будут абузить), ну и зачем нам обычный и перевернутый восклицательный знак? Оставь плюс, минус и цвет. |
восклицательные знаки как раз чтобы было больше похоже на что-то иное чем "заблокировано/разблокировано". Потому что просто плюс и минус - выглядит больше как раз как заблочено/разблочено... |
Ты тут босс, тебе решать. Спасибо за офигенный ПР. |
Та не, решать мейнтейнерам. Как скажут так и будет. |
|
О, это же фича, которой не будут пользоваться!
Добавь куда-то пояснение че вообще значат эти !+ и !-. Можешь сделать, чтобы это выглядело как кнопка, а когда игрок наводит, то появляется hovertext. |
над поискать где в билде есть ховертекст. Эта менюшка же не в наноуи даже, оно просто хардкодом передаётся. |
Мышки не видно, но я навёл на кнопку. |
Сделал всплывающую подсказку что появляется при наведении на профессии что "требуются" или "не требуются".
|
Тут тоже, к слову, скину. |
|
Вот би стать мейнтейнером чтобы мержить всё подряд, особенно то, что улучшает лоупоп экспириенс, в эти сложные для сервера времена... |
Убрал цифры в Дефайн.
code/game/machinery/computer/card.dm
Outdated
| for(var/datum/job/job in SSjob.occupations) | ||
| if(!job) | ||
| continue | ||
| var/list/jobList = list(list("name" = job.title, "type" = job.type, "quota" = job.quota)) | ||
| var/categorized = FALSE | ||
| for(var/jobcat in jobsCategories) | ||
| if(!categorizedJobs[jobcat]) | ||
| categorizedJobs[jobcat] = list("title" = jobcat, "jobs" = list(), color = jobsCategories[jobcat]["color"]) | ||
| var/list/jobs = categorizedJobs[jobcat]["jobs"] | ||
| if(job.title in jobsCategories[jobcat]["titles"]) | ||
| categorized = TRUE | ||
| if(jobcat == "Command") | ||
|
|
||
| if(job.title == "Captain") // Put captain at top of command jobs | ||
| jobs.Insert(1, jobList) | ||
| else | ||
| jobs += jobList | ||
| else // Put heads at top of non-command jobs | ||
| if(job.title in command_positions) | ||
| jobs.Insert(1, jobList) | ||
| else | ||
| jobs += jobList | ||
| if(!categorized) | ||
| categorizedJobs["Miscellaneous"]["jobs"] += jobList |
There was a problem hiding this comment.
я не совсем понимаю нужду генерировать это все на ходу каждый раз при открытии менюшки
у нас разве уже нет готовых листов в нужном порядке? https://github.com/TauCetiStation/TauCetiClassic/blob/25977739f68995654a95b67950a69eb7dff88abb/code/game/jobs/jobs.dm
There was a problem hiding this comment.
Неа, нет готовых листов в нужном порядке.
There was a problem hiding this comment.
а чем command_positions, engineering_positions, security_positions и подобные не подходят?
There was a problem hiding this comment.
Тем, что это лишь часть списка блин
There was a problem hiding this comment.
Там вместе со списком передаётся цвет обрамления:
There was a problem hiding this comment.
Окей, у меня есть идея как чуть сократить код.
There was a problem hiding this comment.
Переделал. Теперь списки почти не составляются. Всё ещё надо их дополнять, но уже меньше проходов и вообще меньше операций.
Описание изменений
Добавил возможность для ГП ставить "лайк" и "дизлайк" нужным и ненужным профессиям:
После чего в лобби, в меню Join, мы можем увидеть какие профессии нужны, а какие нежелательны, по мнению ГП:
Почему и что этот ПР улучшит
Можно будет увидеть какая профессия нужна в раунде по мнению ГП и зайти на неё. Больше работы хорошему ГП. Теперь он будет ещё и опрашивать отделы и следить за тем какие профессии нужны, а какие - нет.
Авторство
AndreyGysev
Чеинжлог
🆑 AndreyGysev